Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.0.0] Move Apple multi arch linking support out of the bazel binary #24273

Merged
merged 4 commits into from
Nov 14, 2024

Conversation

pzembrod
Copy link
Contributor

Remove internal functions for Apple multi arch linking from apple_common and instead expose functions needed by multi arch linking functions in the repo.

… linking_support.bzl (both from builtins_bzl/common/objc/) via apple_common.

This is inpreparation for moving the content of both bzl files to the repo in unknown commit.

PiperOrigin-RevId: 686100272
Change-Id: Iadbe8da6a4335e30daca889cd19afcb627df2b5c
PiperOrigin-RevId: 689003624
Change-Id: I0826d0ddea18b6c4b21e009d93c4de46389031ad
…h_binary_support.bzl and the related providers after their move to rules_apple.

PiperOrigin-RevId: 694055059
Change-Id: Ica3f756f7ec10869cbc7cd475c8d0d962fe7d274
@pzembrod pzembrod requested a review from a team as a code owner November 11, 2024 16:47
@pzembrod pzembrod requested review from comius and removed request for a team November 11, 2024 16:47
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Nov 11, 2024
@iancha1992 iancha1992 changed the title Move Apple multi arch linking support out of the bazel binary [8.0.0] Move Apple multi arch linking support out of the bazel binary Nov 11, 2024
@iancha1992 iancha1992 added this to the 8.0.0 release blockers milestone Nov 11, 2024
@iancha1992 iancha1992 enabled auto-merge November 11, 2024 19:00
@iancha1992 iancha1992 added the team-Rules-CPP Issues for C++ rules label Nov 12, 2024
@iancha1992
Copy link
Member

@pzembrod one of the tests is failing. FYI.

@meteorcloudy
Copy link
Member

The Windows failure was unrelated but due to recent CI upgrade, fixed at #24269

I rebased this PR and the presubmit should pass now.

@iancha1992 iancha1992 added this pull request to the merge queue Nov 14, 2024
Merged via the queue into bazelbuild:release-8.0.0 with commit c877f8a Nov 14, 2024
45 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 14, 2024
@meteorcloudy
Copy link
Member

@keith
Copy link
Member

keith commented Dec 5, 2024

blocked on #24582

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-CPP Issues for C++ rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants